Skip to content

Conversation

@joeizang
Copy link
Collaborator

@joeizang joeizang commented Aug 29, 2025

fixes #6305 290825

@joeizang joeizang self-assigned this Aug 29, 2025
@joeizang joeizang added the WIP label Aug 29, 2025
@netlify
Copy link

netlify bot commented Aug 29, 2025

Deploy Preview for remixproject ready!

Name Link
🔨 Latest commit 83eabf5
🔍 Latest deploy log https://app.netlify.com/projects/remixproject/deploys/68b6d0098b1f350008b74d94
😎 Deploy Preview https://deploy-preview-6320--remixproject.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@joeizang joeizang added ready-to-review PR ready to review and removed WIP labels Sep 1, 2025
@joeizang joeizang requested a review from Aniket-Engg September 1, 2025 14:16
Copy link
Collaborator

@Aniket-Engg Aniket-Engg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I checked functionality wise.

  • There are errors in browser console like Invalid DOM property fill-rule. Did you mean fillRule? , Invalid DOM property clip-rule. Did you mean clipRule? and [TIMEOUT] Timeout for call pinView from sidePanel

  • Icon for restore plugin is appearing like this on my screen:

Screenshot 2025-09-01 at 9 55 02 PM
  • If I open files in editors, that icon starts moving on right and files start wrapping keeping this much right margin from that icon
Screenshot 2025-09-01 at 9 56 31 PM
  • If I move Solidity compiler plugin on right and close it. Plugin icon panel still shows it on right.
Screenshot 2025-09-01 at 9 57 54 PM
  • In such case, If I click on RemixAi assistant, it opens Solidity compiler plugin on right side

@joeizang
Copy link
Collaborator Author

joeizang commented Sep 1, 2025

  • ere are errors in browser console like Invalid DOM property fill-rule. Did you mean fillRule? , Invalid DOM property clip-rule. Did you mean clipRule? and [TIMEOUT] Timeout for call pinView from sidePanel

For point 1, I fixed the regression that was causing the console errors. The last point you made is also addressed by fixing this regression.

The plugin will still show on the right because it is still pinned. when you click the restore icon, it restores it back as pinned. We agreed on the last team call not to close or unpin any pinned plugin that gets closed.

I moved the restore icon to the topbar because the tabs bar has very precise settings that would not be fixed easily.

@joeizang joeizang requested a review from Aniket-Engg September 2, 2025 08:24
Copy link
Collaborator

@Aniket-Engg Aniket-Engg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • On Solidity Compiler left & right, In browser console, I see: compiler-container.tsx:380 Uncaught (in promise) TypeError: Cannot read properties of null (reading 'selectorList') at compiler-container.tsx:380:49 That is not a case on live

  • Icon looks like this on top bar for me:

Screenshot 2025-09-02 at 1 49 25 PM

I think it should be well aligned with other UI elements on top bar

  • If a plugin is hidden on right side and I click on it from left side icon panel, it doesn't appear
  • I think tooltip on X icon should be Hide plugin and while restoring it tooltip on icon can show plugin name if possible like Open RemixAI Assistant plugin

@joeizang joeizang requested a review from Aniket-Engg September 2, 2025 14:29
@ninja-fire ninja-fire self-requested a review September 2, 2025 14:31
@joeizang joeizang merged commit 755b006 into master Sep 2, 2025
32 checks passed
@joeizang joeizang deleted the close-pinned-plugin branch September 2, 2025 14:55
Copy link

@bedroomeyes555 bedroomeyes555 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No direct security risks are apparent in these changes—they are UI/UX improvements for plugin management.
No handling of user credentials, private keys, or sensitive workspace data.
Events and DOM operations are local and do not expose data externally.
Recommendation:
If Remix IDE supports third-party plugins (especially those from remote sources), always ensure: Plugin profile objects are sanitized.
Only trusted plugins can trigger or respond to internal events.
UI state changes cannot be hijacked via crafted plugin data.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-review PR ready to review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

remixAI Assistant removal

5 participants